[선재] WEEK10 Solutions #537
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
haklee
reviewed
Oct 19, 2024
| * brainstorming: | ||
| * preorder traverse | ||
| * | ||
| * n = length of root |
There was a problem hiding this comment.
혹시 length of root은 무엇을 의미하는 것일까요?
HC-kang
reviewed
Oct 19, 2024
| const dfs = (current) => { | ||
| const linkedNode = graph.get(current); | ||
|
|
||
| if (memo[current] || !linkedNode || linkedNode.length === 0) return true; |
There was a problem hiding this comment.
이 부분은 옵셔널 체이닝을 사용하시면 조금 더 간결해질 것 같아요!
if (memo[current] || linkedNode?.length){ ... }
| * brainstorming: | ||
| * preorder traverse | ||
| * | ||
| * n = length of root |
There was a problem hiding this comment.
length of root는 전체 노드의 수를 말씀하시는걸까요?
| @@ -0,0 +1,43 @@ | |||
| /** | |||
| * @description | |||
| * queue의 특성을 활용하여 풀이 | |||
There was a problem hiding this comment.
이 부분에 대해서 혹시 추가 설명을 요청드려도 될까요?
| * space complexity: O(p) | ||
| */ | ||
| var canFinish = function (numCourses, prerequisites) { | ||
| const memo = Array.from({ length: numCourses + 1 }, () => false); |
There was a problem hiding this comment.
와 처음에는 메모가 과연 효과가 있을까..? 생각했는데 몇번을 돌려도 Beats 100%네요 👍
Sunjae95
commented
Oct 19, 2024
| const dfs = (current) => { | ||
| const linkedNode = graph.get(current); | ||
|
|
||
| if (memo[current] || !linkedNode || linkedNode.length === 0) return true; |
Comment on lines
+32
to
+38
| if (answer === null) { | ||
| answer = lists[minIndex]; | ||
| tail = answer; | ||
| } else { | ||
| tail.next = lists[minIndex]; | ||
| tail = lists[minIndex]; | ||
| } |
There was a problem hiding this comment.
@HC-kang
매번 lists의 각 lists의 head중 가장 작은 index의 head를 바꿔주는방법이 queue의 선입선출을 연상되어서 적었습니다. 🙂
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
답안 제출 문제
체크 리스트
In Review로 설정해주세요.